Skip to content

ci(workflow): Add auto-merge notification workflow#548

Merged
Aaron ("AJ") Steers (aaronsteers) merged 2 commits intomainfrom
devin/1747428018-add-automerge-workflow
May 16, 2025
Merged

ci(workflow): Add auto-merge notification workflow#548
Aaron ("AJ") Steers (aaronsteers) merged 2 commits intomainfrom
devin/1747428018-add-automerge-workflow

Conversation

@devin-ai-integration
Copy link
Copy Markdown
Contributor

@devin-ai-integration devin-ai-integration bot commented May 16, 2025

ci(workflow): Add auto-merge notification workflow

Overview

This PR adds a GitHub Actions workflow that adds or updates markdown notices in PR descriptions when auto-merge is enabled or disabled.

Implementation Details

  • Adds a new workflow file that triggers on auto_merge_enabled and auto_merge_disabled PR events
  • Uses the bcgov/action-pr-description-add@v2.0.0 action to modify PR descriptions
  • Adds an IMPORTANT notice when auto-merge is enabled
  • Adds a NOTE when auto-merge is disabled or removed
  • Author can manually clear notices to reset.

References

Link to Devin run

https://app.devin.ai/sessions/78ed5871a67a4474b33d241508af6bdf

Requested by: Aaron ("AJ") Steers (aj@airbyte.io)

Important

Auto-merge enabled.

This PR is set to merge automatically when all requirements are met.

Co-Authored-By: Aaron <AJ> Steers <aj@airbyte.io>
@devin-ai-integration
Copy link
Copy Markdown
Contributor Author

Original prompt from Aaron:

Received message in Slack channel #ask-devin-ai:

@Devin - Please review this failure in PyAirbyte. The intent of this workflow was to add text to the PR when the PR is marked for auto-merge, but it has failed since its introduction. Error message makes no sense. Can you propose a fix or remove the workflow in a new PR? <https://github.com/airbytehq/PyAirbyte/actions/runs/15075086312/job/42380546299?pr=668>

@devin-ai-integration
Copy link
Copy Markdown
Contributor Author

devin-ai-integration bot commented May 16, 2025

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

@github-actions github-actions bot added the ci label May 16, 2025
@aaronsteers Aaron ("AJ") Steers (aaronsteers) marked this pull request as ready for review May 16, 2025 20:43
Copilot AI review requested due to automatic review settings May 16, 2025 20:43
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR introduces a GitHub Actions workflow that updates PR descriptions with markdown notices based on auto-merge status.

  • Adds a new workflow file triggered on auto-merge enable/disable events.
  • Uses the bcgov/action-pr-description-add@v2.0.0 action to add an IMPORTANT notice when auto-merge is enabled and a NOTE when it is disabled.
Comments suppressed due to low confidence (1)

.github/workflows/auto_merge_notification.yml:1

  • Consider revising the comment text to remove the redundant phrasing 'is has'. For clarity, update it to something like: 'When a PR has the auto-merge feature enabled or disabled, this workflow adds or removes warning text at the bottom of the PR description.'
# When a PR is has the auto-merge feature enabled or disabled, this workflow adds or removes warning text at the bottom of the PR description.

Comment thread .github/workflows/auto_merge_notification.yml Outdated
@aaronsteers Aaron ("AJ") Steers (aaronsteers) changed the title ci(workflow): Add auto-merge notification workflow (do not merge) ci(workflow): Add auto-merge notification workflow May 16, 2025
@aaronsteers Aaron ("AJ") Steers (aaronsteers) merged commit 77a68a5 into main May 16, 2025
23 of 24 checks passed
@aaronsteers Aaron ("AJ") Steers (aaronsteers) deleted the devin/1747428018-add-automerge-workflow branch May 16, 2025 20:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants